Skip to content

starship: add v1.25.1#5111

Merged
RikkiButler20 merged 1 commit into
spack:developfrom
mcmehrtens:starship/add-version-1.25.1
Jun 9, 2026
Merged

starship: add v1.25.1#5111
RikkiButler20 merged 1 commit into
spack:developfrom
mcmehrtens:starship/add-version-1.25.1

Conversation

@mcmehrtens

Copy link
Copy Markdown
Contributor
  • Add v1.25.1
  • Drops the cmake requirement since it is not needed for any of the checksummed versions
  • Added more info to license. License hasn't changed since the first tagged release, v0.2.0.

Confirmed all four versions build successfully.

Maintainers: @ebagrenrut

@RikkiButler20 RikkiButler20 changed the title starship: add v1.25.1 and drop cmake requirement starship: add v1.25.1 Jun 8, 2026
@RikkiButler20

Copy link
Copy Markdown
Contributor

@ebagrenrut Can you please review?

@ebagrenrut

ebagrenrut commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@mcmehrtens I'm pretty sure that I added the cmake dependency because starship relies on the Rust cmake crate, which has a dependency on the cmake command being available. Though it's been a while, I have a vague recollection of running into a build failure via Spack CI as a result of having no cmake command in my build image. So though you will find no CMake-related files in the starship source tree, I suspect that the cmake crate is included because some other crate needs cmake. To confirm that cmake really isn't needed, did you confirm that you could build on a system, or in a container, that does not have CMake available?

In fact, the dependency on the cmake create is I think not even a direct dependency of starship itself, but a dependency of a starship dependency (or at least somewhere in the tree of dependencies that cargo pulls in...).

There are very likely other dependencies missing from the starship Spack package that Spack could fulfill, I just didn't encounter them. Because Spack doesn't fully model Rust-based builds, I don't think we currently have a way to ferret out all of the dependencies without brute-force (and possibly laborious) analysis of the full cargo-based build :/

@ebagrenrut

Copy link
Copy Markdown
Contributor

@mcmehrtens At this point, with Spack develop and the checksummed versions of starship, I'm unable to reproduce any issues with cmake, even when in a container with no cmake (neither from spack, nor from the OS). So I approve of this change.

Thanks!

@RikkiButler20 RikkiButler20 merged commit f454715 into spack:develop Jun 9, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants